Skip to content

Conversation

@SampathKumarAmex
Copy link
Contributor


//https://mvnrepository.com/artifact/com.nimbusds/nimbus-jose-jwt
implementation group: 'com.nimbusds', name: 'nimbus-jose-jwt', version: '8.3'
implementation group: 'com.nimbusds', name: 'nimbus-jose-jwt', version: '9.12'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SampathKumarAmex this should not break our existing vulnerable levels. Can you please check vulnerable level using this library and see if that is working as expected

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please provide more info on how to test that? Do you have any documentation covering that part?

It's good to have some test for this if feasible.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SampathKumarAmex we need to start VulnerableApp, go to UI, and test that each level doesn't break.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, we are lacking unit tests and other things right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants